-
Notifications
You must be signed in to change notification settings - Fork 8
feat: Redesign the home page #186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
# Conflicts: # layouts/_default/home.html # layouts/operator/list.html
✅ Deploy Preview for fipguide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
# Conflicts: # assets/sass/styles.scss
I would like to play a bit with the color shema before merging :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the effort. The appearance of the main page is a lot better now. I implemented some improvements and pushed the proposal to another PR: #192. Feel free to have a look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason why you replaced the webp picture with a png?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to have the page as separate page, which is also indexed by Google? Otherwise it might be just a translation key like "wip".
Description
Checklist
The content was modified in the following languages: